Skip to content
This repository has been archived by the owner on Oct 9, 2019. It is now read-only.

item highlight #43

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

item highlight #43

wants to merge 6 commits into from

Conversation

Ledoux
Copy link
Contributor

@Ledoux Ledoux commented Nov 20, 2017

@Atala
given coopcycle/coopcycle-web#140
(will be done also in coopcycle web)

@Ledoux
Copy link
Contributor Author

Ledoux commented Nov 20, 2017

item

@Atala
Copy link
Member

Atala commented Nov 21, 2017

Nice !

I think the grey flash on the non-clicked items is a little bit too much (and maybe an anti-pattern as no action is exerced on these items (I use complicated words to look like an UX designer)).

Could you add a + icon on the menu item line so the user knows it is clickable without hovering it

@Ledoux
Copy link
Contributor Author

Ledoux commented Dec 1, 2017

item

@Ledoux
Copy link
Contributor Author

Ledoux commented Dec 1, 2017

@Atala
ok... seems good for you?
I should begin the same work on coopcycle-web

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants